Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable PWM #254

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Enable PWM #254

wants to merge 1 commit into from

Conversation

clupprich
Copy link

I'm working on a little home automation project and wanted to use RPi4's PWM pins to control a led driver.

With the current system the /sys/class/pwm directory wasn't present, so I googled a bit and found https://elixirforum.com/t/rpi4-hardware-pwm-dtoverlay-need-kernel-module/33264. I think I made the right modifications, it's working for me right now.

@fhunleth
Copy link
Member

fhunleth commented Oct 9, 2024

@clupprich Sorry for the delays!!! I figured that I'd work through some major updates to all Nerves systems first and then get to this. It's taking longer than expected. Suffice it to say, I think your changes are fine, but I'm not sure about enabling unconditionally. Anyone using the PWM pin as a normal GPIO will see their code break. However, maybe we just comment out enabling the pwm dtoverlay in the config.txt with a note. Your other changes will majorly safe the next person time.

Anyway, just wanted to let you know that I hadn't forgotten about this. Once I get the big changes (Erlang, Buildroot, GCC, etc. updates in), I'll get to pushing this PR across the finish line and getting a release out. Thanks for submitting it!

@clupprich
Copy link
Author

@fhunleth No worries at all!

Anyone using the PWM pin as a normal GPIO will see their code break.

Oh, yeah, I didn't think about that (I'm very new to the embedded world). I can update my PR with your suggested changes.

Thanks for maintaining this ecosystem - it's great fun to explore it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants